Skip to content

update cicd - #40

Merged
fanux merged 2 commits into
FullAgent:mainfrom
HUAHUAI23:test.1
Nov 6, 2025
Merged

update cicd#40
fanux merged 2 commits into
FullAgent:mainfrom
HUAHUAI23:test.1

Conversation

@HUAHUAI23

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Nov 6, 2025

Copy link
Copy Markdown

✅ PR Check Results: Passed

Build Checks

Check Status
Lint & Build ✅ Passed
Docker Build ✅ Passed

✨ Great work!

All checks passed successfully. Your PR is ready for review.

Details:

  • ✅ Code quality verified (linting passed)
  • ✅ Build successful
  • ✅ Docker image build verified (linux/amd64)
    Commit: 1abb06d1fe242181ac19618ba3a471f3c9ff2f05
    Branch: test.1

🔗 View Details:

@fanux
fanux merged commit a7b0828 into FullAgent:main Nov 6, 2025
5 checks passed
@HUAHUAI23
HUAHUAI23 requested a review from Copilot November 6, 2025 05:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enhances the Prisma database client configuration to support multi-platform Docker builds by adding explicit binary targets for Alpine Linux architectures, and adds security hardening to the GitHub Actions workflow by sanitizing branch names displayed in PR comments.

  • Added Prisma binary targets for linux-musl-openssl-3.0.x and linux-musl-arm64-openssl-3.0.x to support Alpine Linux builds on AMD64 and ARM64 architectures
  • Added sanitization of branch names in GitHub Actions workflow to prevent Markdown injection attacks
  • Added documentation comment explaining the Prisma binary target configuration

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
prisma/schema.prisma Added binaryTargets configuration to generate Prisma engines for Alpine Linux (musl) on AMD64 and ARM64 platforms
Dockerfile Added comment explaining that Prisma will use the binaryTargets from schema.prisma
.github/workflows/pr-check.yml Added branch name sanitization to escape special Markdown characters before displaying in PR comments

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@HUAHUAI23
HUAHUAI23 deleted the test.1 branch November 6, 2025 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants